Skip to content

beholder: add telemetry LogLevel config option for log streaming #1591

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
pavel-raykov merged 3 commits into from
Oct 9, 2025
Merged

beholder: add telemetry LogLevel config option for log streaming #1591

pavel-raykov merged 3 commits into from
Oct 9, 2025

Conversation

@pkcll
Copy link
Contributor

@pkcll pkcll commented Oct 8, 2025 •
edited
Loading

beholder: add Telemetry.LogLevel config option for log streaming

Related PR smartcontractkit/chainlink#19775

@github-actions
Copy link

github-actions bot commented Oct 8, 2025 •
edited
Loading

⚠️ API Diff Results - Breaking changes detected

📦 Module: github-com-smartcontractkit-chainlink-common

🔴 Breaking Changes (1)

pkg/loop (1)
  • NewOtelLogger — Type changed:
- func(go.opentelemetry.io/otel/log.Logger) (github.com/smartcontractkit/chainlink-common/pkg/logger.Logger, error)
+ func(go.opentelemetry.io/otel/log.Logger, go.uber.org/zap/zapcore.Level) (github.com/smartcontractkit/chainlink-common/pkg/logger.Logger, error)

📄 View full apidiff report | 📚 Learn about apidiff

@pkcll pkcll mentioned this pull request Oct 8, 2025
Merged
@pkcll pkcll force-pushed the infoplat-2449-otelzap-logger-add-telemetry-log-level branch from e4615c9 to ab8d58b Compare October 8, 2025 03:30
@pkcll pkcll force-pushed the infoplat-2449-otelzap-logger-add-telemetry-log-level branch from 3d3cad8 to b811a20 Compare October 8, 2025 03:37
@pkcll pkcll marked this pull request as ready for review October 8, 2025 03:41
@pkcll pkcll requested review from a team as code owners October 8, 2025 03:41
@pkcll pkcll changed the title beholder: add Telemetry.LogLevel config option for log streaming beholder: add telemetry LogLevel config option for log streaming Oct 8, 2025
@pkcll pkcll mentioned this pull request Oct 8, 2025
Merged
jmank88
jmank88 reviewed Oct 8, 2025
View reviewed changes
@pkcll pkcll requested review from bolekk and patrickhuie19 and removed request for albert597 October 9, 2025 16:02
@pavel-raykov pavel-raykov enabled auto-merge (squash) October 9, 2025 16:17
@pavel-raykov pavel-raykov merged commit 1bb9638 into main Oct 9, 2025
20 of 21 checks passed
@pavel-raykov pavel-raykov deleted the infoplat-2449-otelzap-logger-add-telemetry-log-level branch October 9, 2025 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jmank88 jmank88 jmank88 approved these changes

@patrickhuie19 patrickhuie19 patrickhuie19 approved these changes

@kirqz23 kirqz23 kirqz23 approved these changes

@4of9 4of9 Awaiting requested review from 4of9

@engnke engnke Awaiting requested review from engnke

@fouadkada fouadkada Awaiting requested review from fouadkada

@hendoxc hendoxc Awaiting requested review from hendoxc

@krehermann krehermann Awaiting requested review from krehermann

@bolekk bolekk Awaiting requested review from bolekk

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@pkcll @jmank88 @patrickhuie19 @kirqz23 @pavel-raykov